[17.0][MIG] crm_salesperson_planner: Migration to 17.0 #585
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[UPD] Update crm_salesperson_planner.pot [UPD] README.rst
- Missing column renaming for not collapsing - Missing weekday conversion
BernatObrador
force-pushed
the
17.0-mig-crm_salesperson_planner
branch
from
61ce6e8
to
815e231
Compare
peluko00
reviewed
Jul 22, 2024
BernatObrador
force-pushed
the
17.0-mig-crm_salesperson_planner
branch
2 times, most recently
from
0bcfd6c
to
42daedb
Compare
Currently translated at 100.0% (196 of 196 strings) Translation: crm-16.0/crm-16.0-crm_salesperson_planner Translate-URL: https://translation.odoo-community.org/projects/crm-16-0/crm-16-0-crm_salesperson_planner/es/
Changes done: - Removed the (unnecessary) recurrence field from templates. - Recurrence fields (from calendar.event) are set as store=True. - Create an onchange to avoid inconsistent values by UX. - Modified the form view to make it more functional (similar to calendar.event). - Consider in _get_max_date() if a deadline (until) has been set. TT48238
… usable and with simplified code. TT48238
Currently translated at 100.0% (186 of 186 strings) Translation: crm-16.0/crm-16.0-crm_salesperson_planner Translate-URL: https://translation.odoo-community.org/projects/crm-16-0/crm-16-0-crm_salesperson_planner/es/
Currently translated at 100.0% (186 of 186 strings) Translation: crm-16.0/crm-16.0-crm_salesperson_planner Translate-URL: https://translation.odoo-community.org/projects/crm-16-0/crm-16-0-crm_salesperson_planner/it/
BernatObrador
force-pushed
the
17.0-mig-crm_salesperson_planner
branch
from
42daedb
to
ccf6f8f
Compare
ppyczko
approved these changes
Jul 22, 2024
BernatObrador
force-pushed
the
17.0-mig-crm_salesperson_planner
branch
from
ccf6f8f
to
0e43619
Compare
|
/ocabot migration crm_salesperson_planner |
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at a48db4d. Thanks a lot for contributing to OCA. ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Module migrated to version 17.0
cc https://github.com/APSL 158506
@miquelalzanillas @lbarry-apsl @mpascuall @peluko00 @javierobcn @ppyczko please review
Issue #525